a11y: Emit row-deleted signal when rows get deleted

Instead of in the old callbacks.

a11y: Revert 22a47c3dbd7875af10df58feedd81d7b661222b6

The fix is wrong on multiple levels. I'll add a proper fix soon.

https://bugzilla.gnome.org/show_bug.cgi?id=548782

treeview: Don't use signals in a11y row expansion

Instead, add a function and call it directly.

a11y: Add _gtk_tree_view_accessible_add_state()

... and _gtk_tree_view_accessible_remove_state(). They should be called
by gtktreeview.c to notify about changes of state for a row.

a11y: Split out function

I want to use that function when sending notifications. See next
commits.

a11y: Readd FOCUSABLE state

The state was lost in commit 08d86fabc4237092bf04a97795498f2c45575085.

Also order the states that are always set alphabetically.

a11y: Remove old notify about showing/visible

As cells are always visible and showing, there is no need to do complex
notification mechanisms anymore.

a11y: Add _gtk_cell_accessible_state_changed()

It's supposed to replace the old state_set calls.

a11y: Split out renderer<->atk state matching

We now keep a state map. This makes adding new states easier, and
doesn't introduce bugs with state change emissions.

a11y: Don't duplicate information anymore

We do provide the expandable and expanded information via
GtkCellRendererState now so there is no need to use extra flags.

a11y: Set new treeview flags

treeview: Set the new flags when rendering the cell

cellrenderer: Add EXPANDED and EXPANDABLE states

These make more sense as flags than they do as properties of the cell
renderer.

tests: Add a test for latest commit

widget: Don't change sensitive state with insensitve parent

This way child.set_sensitive(True) with an insensitive parent doesn't
cause child to become sensitive.

fontchooserwidget: Handle dpi < 0

Just use 96.0

Add new symbols

iconhelper: treat the empty string as a NULL icon name

Treat the empty string as if it was a NULL icon name instead of
rendering GTK_STOCK_MISSING_IMAGE.
This used to be the previous behavior of GtkCellRendererPixbuf and
applications might still rely on it.

cellrendererpix: save the icon size locally instead of in the helper

We need to save the icon size locally for GtkCellRendererPixbuf, since
its properties are set again every time it's drawn (resetting the icon
size on the helper by means of _gtk_icon_helper_clear() each time).

menuitem: cleanup arrow-spacing fetch code

Refactor get_arrow_size() to return the spacing as well, since basically
we always need both.

menuitem: share code between HFW and non-HFW height requests

The code is very similar; having two slightly different code paths is
bad and can lead to bugs. Refactor the code to use the same height
request function.

menuitem: properly respect padding when drawing the separators

Use the padding when calculating the draw coordinates of the separator.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

menuitem: force odd height if we're not using wide separators

So that the 1px line can receive the same amount of space above and
below.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

menuitem: don't request twice the padding for separator menu items

Padding was requested twice for separator menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=666242

demo: Add test for hide-titlebar-when-maximized

Add an additional option to the appwindow demo to test the new
gtk-hide-titlebar-when-maximized setting.

https://bugzilla.gnome.org/show_bug.cgi?id=665616

window: Add hide-titlebar-when-maximized property

For maximized windows, titlebars cannot be used to reposition or
scale the window, so if an application does not use it to convey
useful information (other than the application name), the screen
space occupied by titlebars could be put to better use.
Add a new window property which requests from the window manager
to hide titlebars when windows are maximized to account for this.

https://bugzilla.gnome.org/show_bug.cgi?id=665616

scale: remove unused variable

widget: Synthesize crossing only if sensitivity changed while propagating state

Add has-origin property for GtkScale

If the scale has an origin (it will have one by default), GtkRange will
render the two sides before/after the current value with different style
classes, making it possible for themes to use different colors and
properties for the two areas.
This was possible in GTK 2 with style details, but got lost during the
road to 3.0.

https://bugzilla.gnome.org/show_bug.cgi?id=665140

wayland: Add dummy implementation of the wl_data_device functionality

Setup listener functions for the drag and drop events as well as the selection
events. Then create and save a data structure representing the data offer from
the other client.

wayland: Add an implementation for GdkAtom

This implementation is based on a hash table. The hard coded GtkSelection
atoms are preloaded into the hash table at the correct values. User generated
atoms start after the last preloaded atom.

wayland: Get the data device during input device setup

The data device is a associated with the input device - fetch the data device
from the data manager for the current input device.

wayland: Retrieve the data device manager object

The data device manager is a global object that provides the support
infrastructure around data devices. These data device objects are the basis
for handling drag and drop as well as selections in Wayland.

wayland: Don't use deprecated function: gdk_window_get_pointer

Replace it with gdk_window_get_device_position.

Updated Slovak translation

print dialog: Fix authentication logic

Reset state of CUPS requests correctly during authentization and
check CUPS requests for errors.
Don't initialize variables holding password and username with empty
string (#664640).

Fix nicks for the window class enum

As pointed out in bug 665999, these were just not right.
Before this commit, the nicks were 'output' and 'only'.
After this commit, they are 'input-output' and 'input-only'.

Fix set_focus_child handling during focus changes

In many cases we used to set focus_child to NULL all the way up
to the top and then to the right value, even if there was
a common ancestor, meaning these see a temporary NULL value for
focus_child. Only when the new focus widgets direct parent was
in the previous ancestor list did we stop early.

This fixes that by always stopping propagation when reaching
the common ancestor.

GtkFrame: Fix a casting in the compute_child_allocation() method

The casting used to calculate the child allocation is confusing MAX().
As a result, width and height end up with negative values, which makes no sense.

https://bugzilla.gnome.org/show_bug.cgi?id=666109

themingengine: split a GtkThemingBackground object

Allows easier modification and general sanity of the CSS background
rendering code.

image: take misc padding into account when calculating x/y draw coords

This fixes the misc-alignment.ui reftest.

image: add gtk_image_get_preferred_size()

Add a function to query the preferred size of the GtkImage, and use it
in the get_preferred_width/height implementations.

treeview: Ensure selection's changed signal

There was a corner case where the changed signal was not emitted.
If rows were built like this:
  1    (not selected)
  + 2  (selected)
  + 3  (not selected)
And row 1 was removed, no signal would be emitted.

I like it when writing tests actually finds bugs that have been around
since 2003 - introduced by 4a03ea2334bde7d2e15d0933c5a78f5a2338a854
actually. :)

treeview: Don't try to optimize away selection_changed signal

We get certain cases, in particular with SELECTION_MULTIPLE, where we
cannot figure out in advance of real_set_cursor() if the selection will
actually change.

tests: Add select/unselect calls

treeview: Add a CURSOR_INVALID flag to set_cursor_row()

This is necessary so the code doesn't attempt to do things with the
previous cursor, which currently cause crashes.

treeview: Replace lots of boolean args with flags

I want to add more and 3 boolean args is essentially unreadable code.

tests: Add sanity checks for selection

We want to know that the selection emits "changed" when it changes.

docs: Update code demo for newer glib

tests: Dump the operation that is performed to stderr

tests: Make treechanging test run as fast as possible

treeview: Handle the case where the cursor row gets deleted

Previously, the cursor would just become invalid, which used to
reselect the first row in the treeview later on (without a
cursor-changed signal). This leads to a crash now with the recent
refactorings.

The patch is longer than I'd like it to be, but the situation is kinda
complicated, because we want to make sure to move the cursor to a good
row. It works like this:
1) From the deleted row, search forward for the first row that is not
   going to be deleted and is not a separator.
2) If no such row exists, search backwards for a row that is not a
   separator.
3) If no such node exists, clear the cursor.

treeview: Replace cursor handling by keeping the node

Previously the code used a GtkTreeRowReference, which was (a) less
performant and more importantly (b) hiding errors.
The errors being hidden were the referenced row becoming invalid or
collapsed, because such rows would not be valid cursor rows and it would
be necesary to select a new row and emit a "cursor-changed" signal.

So if a crash bisects down to this commit, it is very likely that the
cursor has not correctly been updated and the cursor row is invalid.

tests: Put random number into local variable

This way, we can see what function had previously been called when the
checks fail.

treeview: Use _gtk_rbtree_contains() in a11y code

rbtree: Add _gtk_rbtree_contains()

It's useful in a bunch of places, so split it out

Fix move-resize emulation with XI2

https://bugzilla.gnome.org/show_bug.cgi?id=650943

wayland: Load cursors from wayland directory under XDG_DATA_DIRS

This means we don't have to have hardcoded "/usr/share/wayland" to find the
cursors.

This change also fixes up the warning messages for when loading fails.

Updated Breton translation

Updated Breton translation

Revert 3fd7dbc94074826755da7fc3237e439ba6e81ada

Calling gdk_keymap_add_virtual_modifiers causes _all_ virtual
modifiers to be added, which causes problem when they are co-located
on the same real modifier (as Super and Hyper often are). Effectively,
this made it impossible to enter key combinations involving Super,
since they all turn into Super+Hyper.

Test some virtual modifiers

This adds some accelerators involving Super and Hyper to testmerge.
This helps in testing our accel matching for those.

print to file: fixed lost filename in gui

Stop widget creating an event before updating both directory and
filename values.

https://bugzilla.gnome.org/show_bug.cgi?id=603823

overlay: Properly allocate the child upon addition

This ensures that after addition the child gets assigned the proper
allocation and the child window is properly shown/hidden.

overlay: Move code

Move code for child allocation into the child allocation function. Don't
keep it in the overlay allocation code.

See the next commit for why this is useful.

overlay: Create child window before setting parent

Otherwise, the window will not be used when set_parent() maps the child.

progressbar: Set state flags on draw

We were missing setting state flags on the style context before drawing.

widget: Unset window-unfocused in gtk_widget_unparent()

Widgets without a parent aren't inside a toplevel window so we must remove
window-unfocused as it doesn't make sense outside a toplevel.

https://bugzilla.gnome.org/show_bug.cgi?id=661428

treeview: Fix crash when scrolling to end

This broke in a4630d0e7b233479825b059e0df0e6d65b0e6734

https://bugzilla.gnome.org/show_bug.cgi?id=665741

window: copy back window surface data when the implicit paint is flushed

When an implicit paint is flushed during expose, e.g. because a
non-double buffered widget is painting, make sure to copy the existing
data from the window surface we rendered before flushing back to the
paint surface, instead of using an empty base.
Code was already handling that (and said so in the comment), but only
when no implicit paint was used at all, and not in the case when it's
flushed mid-expose.

window: trivial cleanup

window: remove unused arguments and variables

Trivial cleanup, no functional change.

modifierstyle: Fix compile errors

Introduced in 7bd2446801ffd0fc2ac5cc98fcd675ac4f3935d6

Fix demo to use a "normal" transparent value now that it works.

The last commit fixed this, so lets have a sane value in the demo code.

Remove broken optimization in style modifier overrides

We used to only set the override color or font if it was different
than the existing value. However, that means you can't change it to
an override that is the same as the default value for the property.

With this fixed you can e.g. override with a color of 0,0,0,0 which
you couldn't before.

Try to fix linking

We need to link against gdk against pangoft2 on X11.
Also add Requires.private to gdk-3.0.pc, and try to clean
things up some.

https://bugzilla.gnome.org/show_bug.cgi?id=665326

gtk-demo: Add test of transparent GdkWindows

This shows a semi-transparent shadow from a GtkOverlay window

Remove an unused variable

gtk,notebook: Invalidate the gap side when reordering tabs

This narrow area is invalidated so the gap follows the tab
being reordered.

notebook: Make the tab reordering dnd window transparent

statusicon: fix the build on !X11

Uninevitable fallout from my last patch to GtkStatusIcon. Thanks to Rob
Bradford for testing this.

widget: Update Pango context on state change

The layout depends on the state flags, so it must be updated when they
change.

label: Clear layout on state change

Selected or active text might be rendered differently.

label: Move callback to new stuff

We want to react to state flags changing, not to the (deprecated) state
type changing.

wayland: Port from wl_shell to wl_surface_shell

Previously all the commands that acted on the shell took the surface that was
to be acted on as parameter. Now we retrieve an object from the shell that
represents its state for the surface. With that wl_shell_surface object we can
then call methods on that.

Signed-off-by: Kristian Høgsberg <krh@bitplanet.net>

stylecontext: always add the ARROW style class when rendering an arrow

When gtk_render_arrow() is called, always add an ARROW style class to
the GtkStyleContext before rendering, so themes can specify a different
color for it.

https://bugzilla.gnome.org/show_bug.cgi?id=665420

stylecontext: add GTK_STYLE_CLASS_ARROW

For e.g. submenu items arrow in GtkMenuItems.

https://bugzilla.gnome.org/show_bug.cgi?id=665420

statusicon: port to GtkIconHelper

Just always render the pixbuf ourselves and set it on the GtkImage in
the X11 case.
Code for other backends was already rendering the pixbuf manually before
translating it into a native type, so this greatly simplifies the code.

https://bugzilla.gnome.org/show_bug.cgi?id=665409

Merge branch 'transparent-windows'

Make window handle transparent

Instead of painting the window background on the grip_window we now
only paint it on the GtkWindow->window, and we make the grip_window
have a transparent background.

We can't really make transparent window handle background optional
via css atm, because the handle color is actually based on the
background color, so if that is set to transparent we won't draw
anything.

gdk: gdk_window_get_update_area don't remove alpha covered areas

gdk_window_get_update_area is supposed to get the area where things
need painting, and remove them from the update areas. However, if
some area is covered by other windows with an alpha background we
can't just expect whatever the app choses to render in the update
area as correct, so we don't actually remove these areas, meaning
they will get correctly rendered when we get to the expose handlers.

gdk: gdk_window_move_region

gdk_window_move_region doesn't move children, so we can't copy
transparent child window regions with copyarea, so we remove these
from the copy region.

gdk: Fix alpha window background paintings on non-implicit paints

gdk: Remove unused _gdk_window_calculate_full_clip_region

Handle has_alpha_background for parent-relative backgrounds

entry: avoid a mismatched cairo_save/restore

Leftover from GtkIconHelper migration.

Remove unused alignment from GtkTooltip.

Plug tiny leak in the css parser

wayland: Update to recent API changes

GtkIconView: fix a memory corruption in the drag-n-drop code

You shall free with g_slice_free() what you allocate with
g_slice_new().

https://bugzilla.gnome.org/show_bug.cgi?id=665338